444 verify git link #452
444 verify git link #452
Conversation
|
|
||
| def __init__(self, file_info): | ||
| super().__init__(file_info) | ||
| self.check_aprb = FindDefSld(file_info=file_info, key_slide="Апробация") |
There was a problem hiding this comment.
Создайте задачу на сохранение "важных" слайдов в структуре проверяемого файла (чтобы например после проверки FindDefSld в объекте появился соответствующий слайд / ссылка на него, чтобы в дальнейшем можно было его переиспользовать, если такое возможно)
| found_repo = re.findall(self.pattern_for_repo, string_from_text) | ||
|
|
||
| if not found_repo: | ||
| return answer(True, 'Нечего проверять!') |
There was a problem hiding this comment.
На данный момент проверка сделана таким образом, что отсутствие ссылок на репозитории не является недочетом. Оценивается только их корректность (если они есть).
Уточните, пожалуйста, является ли наличие ссылок на репозитории необходимым условием для прохождения именно этой проверки?
| self.deep_check_repo(i, link) | ||
| except (requests.exceptions.SSLError, requests.exceptions.ConnectionError): | ||
| self.wrong_repo_ref.append(i[0]) | ||
| if self.wrong_repo_ref and not self.empty_repo_ref: |
There was a problem hiding this comment.
С этим каскадом условий сразу два момента:
- можно попробовать улучшить читаемость, сделав на wrong_repo_ref и empty_repo_ref проверки раздельно (например, есть пустые? хорошо, просто плюсуем к результирующей строке, что такие есть)
- в 49-54 строках есть условия на Апробоцию: получается так, что результат проверки Апробации вообще никак не сказывается на выходе, если этот каскад проходит в else (попробуйте прогнать презентацию без ссылок и выведет, что все ок)
Кажется, что в обоих случаях будет полезно отлавливать, что результат будет false. Возможно инициализация check_result = True в начале поможет с этим, но может придет в голову что-то другое
There was a problem hiding this comment.
Новый вариант:
fd9dab0
Улучшила читаемость
Поправила ошибку, из-за которой файл "Апробация" не обнаруживался.
Первый вариант проверки слайдов на корректность репозиториев.
Реализованы:
в слайде "Апробация"
Поиск репозиториев github, gitlab, bitbucket.
Поиск некорректных выражений вместо ссылок, например "(github)" или "gitlab"
Отсутствие ссылок при упоминании репозитория .
Во всем проекте:
Репозитории "github" проверяются на существование, пустоту (парсингом), и на приватность.
Репозитории "gitlab" проверяются на существование и приватность.
Репозитории "bitbucket" проверяются на существование, из проверки на приватность исключены, поскольку недоступны в России.
Сейчас для проверки репозитория "gitlab" использован личный токен, нужна консультация, как подкорректировать.
Для более четкой проверки всех репозиториев на пустоту было решено использовать отдельную проверку.